Skip to content

Conversation

@rogernelson
Copy link
Contributor

Build Fix: Ensure python wheels are build with RV's python

Summarize your change.

When building wheels with python, CMake would use whichever Python version it found first, which was usually your system python. This poses a problem when its a different version that RV's python if there are any compiled libraries built (like with opentimelineio for example), since when you try to import the package it will look module built against the matching version of python.

To fix this issue, we will pass the python executable path to pip so we built it with RV's python. We will also use --no-cache-dir to ensure we always built the package. Even if our python version does have a cached version available, we might be building it with different compilers or different versions of shared libraries, so let's not take that chance and just force pip to always build with RV's python. We will also use and --force-reinstall so we will always build OTIO when we build Python to ensure no underlying changes to RV's python make this issue reappear.

Describe the reason for the change.

Build fix

Describe what you have tested and on which operating system.

Reproduced the issue on Mac OS, and verified the fix

Add a list of changes, and note any that might need special attention during the review.

If possible, provide screenshots.

@pbergeron-adsk pbergeron-adsk self-requested a review December 1, 2025 16:25
Copy link
Contributor

@pbergeron-adsk pbergeron-adsk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for fixing this

@rogernelson rogernelson enabled auto-merge (squash) December 1, 2025 16:28
@rogernelson rogernelson disabled auto-merge December 1, 2025 16:30
@rogernelson rogernelson force-pushed the dev/nelsonr/python_cache branch from 5443353 to 2448510 Compare December 1, 2025 16:50
Copy link
Contributor

@bernie-laberge bernie-laberge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rogernelson rogernelson enabled auto-merge (squash) December 1, 2025 18:19
@rogernelson rogernelson merged commit b5fa395 into AcademySoftwareFoundation:main Dec 1, 2025
19 checks passed
@rogernelson rogernelson deleted the dev/nelsonr/python_cache branch December 1, 2025 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants